Skip to content

feat(diagnostics): Add diagnostics buckets setup#265

Merged
andrewazores merged 2 commits intocryostatio:mainfrom
Josh-Matsuoka:diagnostics-buckets
Oct 3, 2025
Merged

feat(diagnostics): Add diagnostics buckets setup#265
andrewazores merged 2 commits intocryostatio:mainfrom
Josh-Matsuoka:diagnostics-buckets

Conversation

@Josh-Matsuoka
Copy link
Contributor

Fixes #260

Adds the diagnostics buckets (threaddumps, heapdumps) to the CRYOSTAT_BUCKETS environment variable for cryostat storage, as well as adding the STORAGE_BUCKETS_THREAD_DUMPS_NAME and STORAGE_BUCKETS_HEAP_DUMPS_NAME environment variables to the cryostat container. Adjusts the tests accordingly.

To test:

  • Spin up a crc cluster or any other test environment
  • helm install --set core.discovery.kubernetes.enabled=true --set core.discovery.kubernetes.namespaces='{default}' --set core.image.repository=quay.io/jmatsuok/cryostat cryostat ./charts/cryostat/
  • install a sample application that uses the cryostat agent (heap dumps require an agent connection)
  • Test the diagnostics functionality, make sure that thread and heap dumps can be triggered and downloaded, etc.
  • Check the environment variables for the cryostat container, make sure that the STORAGE_BUCKETS environment variables are set.
  • Check the environment variables for the storage container, make sure that CRYOSTAT_BUCKETS includes threaddumps and heapdumps

@Josh-Matsuoka Josh-Matsuoka added feat New feature or request safe-to-test labels Oct 3, 2025
andrewazores
andrewazores previously approved these changes Oct 3, 2025
@andrewazores andrewazores merged commit b16c3b5 into cryostatio:main Oct 3, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature or request safe-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request] CRYOSTAT_BUCKETS environment variable should include Thread and Heap Dump buckets

2 participants